Skip to content

Add Cursor Pagination to listUsers #487

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
scazan wants to merge 7 commits into
base: dev
Choose a base branch
from
Open

Add Cursor Pagination to listUsers #487

scazan wants to merge 7 commits into from

Conversation

@scazan
Copy link
Collaborator

@scazan scazan commented Jan 22, 2026

Adds in cursor-based pagination to the listUsers endpoint for supporting many more users. Makes sure that pagination still works with sorting and search

@scazan scazan changed the base branch from dev to listUsers-search January 22, 2026 22:15
@scazan scazan marked this pull request as ready for review January 22, 2026 22:17
@scazan scazan force-pushed the listUser-cursor-pagination branch from 391fea9 to 8c48b9a Compare January 22, 2026 22:24
@scazan scazan marked this pull request as draft January 22, 2026 22:25
@scazan scazan marked this pull request as ready for review January 22, 2026 22:45
@scazan scazan force-pushed the listUser-cursor-pagination branch from 9c95c02 to e72136f Compare January 23, 2026 00:32
valentin0h
valentin0h requested changes Jan 23, 2026
View reviewed changes
Copy link
Collaborator

@valentin0h valentin0h left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a small issue with the sorting.

@scazan scazan force-pushed the listUser-cursor-pagination branch from d7cf1af to d7c0764 Compare January 23, 2026 19:09
Base automatically changed from listUsers-search to dev January 23, 2026 19:20
@scazan scazan force-pushed the listUser-cursor-pagination branch from d7c0764 to fe37251 Compare January 23, 2026 19:24
@scazan scazan requested a review from valentin0h January 23, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@valentin0h valentin0h Awaiting requested review from valentin0h

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@scazan @valentin0h